Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multicast address bugfix and improvements #978

Closed
wants to merge 3 commits into from
Closed

Conversation

ywc689
Copy link
Collaborator

@ywc689 ywc689 commented Jul 10, 2024

  1. fix hw multicast address sync problems
  • Neglected the fact that multicast IPv4/IPv6 address can be mapped to one multicast hw address, and a lower dpvs port may have multiple upper ports (such as vlan).
  • Multicast hw addresses could sync from kni multiple times or be deleted by mistake.
  • Interferences of linked down kni devices.
  1. Add supports for getting dpvs multicast via dpip maddr.

  2. Fix github CI problem caused by outdated Node.js.

It fixes following problems in Previous implementation.
1. Neglected the fact that multicast IPv4/IPv6 address can be mapped to one multicast hw address,
   and a lower dpvs port may have multiple upper ports (such as vlan).
2. Multicast hw addresses could sync from kni multiple times or be deleted by mistake.
3. Interferences of linked down kni devices.

Signed-off-by: ywc689 <[email protected]>
@ywc689 ywc689 changed the base branch from master to devel July 10, 2024 03:47
@ywc689 ywc689 added the pr/to-review-codes review codes line by line and check if problem exists. label Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. labels Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. labels Jul 10, 2024
@ywc689 ywc689 added the pr/to-test-codes Compile and test the patch of pr to verify if it works. label Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. pr/to-test-codes Compile and test the patch of pr to verify if it works. labels Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. labels Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. labels Jul 10, 2024
@ywc689 ywc689 added pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-review-codes review codes line by line and check if problem exists. labels Jul 10, 2024
@ywc689 ywc689 closed this Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/to-review-codes review codes line by line and check if problem exists.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant